Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unboxing the embedded cluster #4951

Merged
merged 25 commits into from
Oct 23, 2024
Merged

Conversation

miaawong
Copy link
Member

@miaawong miaawong commented Oct 15, 2024

What this PR does / why we need it:

This PR creates a new unboxing experience for embedded cluster that guides the users on how to setup and deploy the application.

https://app.shortcut.com/replicated/story/108964/redesign-implementation-for-the-tls-warning-page-unboxing-embedded-cluster
https://www.loom.com/share/7c1ae5ff58084674b580541e42c88662
[sc-108964]

Which issue(s) this PR fixes:

Does this PR require a test?

Does this PR require a release note?

Added a guided installation UI for unboxing the Embedded Cluster

Does this PR require documentation?

@miaawong miaawong added the type::feature New feature or request label Oct 15, 2024
@miaawong miaawong force-pushed the miawong/unboxing-the-embedded-cluster branch from 5bc2259 to fd139e6 Compare October 15, 2024 15:24
@miaawong miaawong requested a review from sgalsaleh October 15, 2024 16:02
@miaawong miaawong marked this pull request as ready for review October 15, 2024 16:02
@@ -55,6 +58,7 @@ function PreflightResultPage(props: Props) {
// TODO: remove this once everything is using react-query
// componentWilUnmount
useEffect(() => {
props.setCurrentStep(3);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this intentional?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes it is. it sets the sidebar step.

kurl_proxy/cmd/main.go Outdated Show resolved Hide resolved
@miaawong miaawong force-pushed the miawong/unboxing-the-embedded-cluster branch from f8e4eca to 17fd68c Compare October 22, 2024 14:02
@miaawong miaawong requested a review from sgalsaleh October 23, 2024 18:59
@miaawong miaawong merged commit a57ff2d into main Oct 23, 2024
122 checks passed
@miaawong miaawong deleted the miawong/unboxing-the-embedded-cluster branch October 23, 2024 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type::feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants